Skip to content

Conversation

@MichaelScofield
Copy link
Contributor

Which issue does this PR close?

Closes #.

Rationale for this change

Some down-stream users of arrow-flight are using tonic newer version (0.13) already, better to make arrow-flight upgrade to tonic 0.13 as well.

What changes are included in this PR?

Simply update tonic version to 0.13 in arrow-flight crate, and re-generate the proto, too.

Are there any user-facing changes?

no

@github-actions github-actions bot added arrow Changes to the arrow crate arrow-flight Changes to the arrow-flight crate labels May 13, 2025
@alamb
Copy link
Contributor

alamb commented May 13, 2025

@alamb alamb reopened this Jul 1, 2025
@alamb alamb closed this in #7839 Jul 16, 2025
alamb added a commit to Samyak2/arrow-rs that referenced this pull request Jul 16, 2025
# Which issue does this PR close?

- Related to apache#7395 
- Closes apache#7495
- Closes apache#7377

# Rationale for this change

Let's update tonic to the latest

Given the open and unresolved questions on @rmn-boiko's PR
apache#7377 from @Xuanwo and @sundy-li,
I thought a new PR would result in a faster resolution.

# What changes are included in this PR?

This PR is based on apache#7495 from
@MichaelScofield -- I resolved some merge conflicts and updated
Cargo.toml in the integration tests

# Are these changes tested?

Yes, by CI
# Are there any user-facing changes?
New dependency version

---------

Co-authored-by: LFC <[email protected]>
@MichaelScofield MichaelScofield deleted the MichaelScofield-patch-1 branch August 5, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrow Changes to the arrow crate arrow-flight Changes to the arrow-flight crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants